Skip to content

Conversation

@climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Feb 11, 2026

Description

  1. Move S4 and Gaea C5 from tier 1 to tier 2.
  2. Clean up all site config modules.yaml files: remove unnecessary include:python. This is a relic from the past when we used external Python installations with Spack and had exclude:python in configs/common/modules.yaml. That entry has been removed a long time ago, and the include:python statements are no longer necessary.
  3. Bug fix in util/nrl/batch_install.sh: When spack creates a bootstrap mirror, it populates the spack scope with compilers and packages it finds, which can create problems later. Remove these files.

Dependencies

None

Issues addressed

Working toward #1883

Applications affected

None

Systems affected

S4, Gaea C5

Testing

  • CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
    • GitHub actions CI tests pass
    • GitHub actions CI tests do not pass (provide explanation)
    • GitHub actions CI tests skipped (provide explanation if necessary)
  • New tests added: List and describe any new tests added to GitHub actions
    • ...
  • Additional testing: Add information on any additional tests conducted
    • Built all spack-stack environments on Atlantis and ran NEPTUNE regression tests against them

Checklist

  • This PR addresses one issue/problem/enhancement or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.
  • All necessary updates to the documentation (spack-stack wiki) will be made when this PR is merged

@climbfuji climbfuji self-assigned this Feb 11, 2026
@climbfuji climbfuji changed the title Move S4 and Gaea C5 from tier 1 to tier 2; clean up site config modules.yaml files Move S4 and Gaea C5 from tier 1 to tier 2; clean up site config modules.yaml files; bug fix in util/nrl/batch_install.sh Feb 11, 2026
@climbfuji climbfuji marked this pull request as ready for review February 11, 2026 15:13
@climbfuji climbfuji enabled auto-merge (squash) February 11, 2026 19:18
@AlexanderRichert-NOAA
Copy link
Collaborator

What are the impetus and implications for moving Gaea C5 to tier 2 status?

@climbfuji
Copy link
Collaborator Author

What are the impetus and implications for moving Gaea C5 to tier 2 status?

EPIC management decided (in accordance with whom?) last year that C5 would no longer be supported as a tier 1 platform. In fact, the last release(s) of spack-stack weren't deployed there. Gaea C6 remains fully supported.

@rickgrubin-noaa knows more.

@AlexanderRichert-NOAA
Copy link
Collaborator

Ok fair enough, thanks

@climbfuji climbfuji merged commit e653daf into JCSDA:develop Feb 12, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in spack-stack-2.1.x (2026 Q1) Feb 12, 2026
@climbfuji climbfuji deleted the feature/module_include_python_cleanup_move_s4_and_gaea_c5 branch February 12, 2026 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants